-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #5791: Implementation of a command line for rudder agent #1
Fixes #5791: Implementation of a command line for rudder agent #1
Conversation
25e26f9
to
70a3e3e
Compare
@@ -0,0 +1,3 @@ | |||
#!/bin/sh | |||
# @description re-enable a disabled rudder-agent | |||
rm -f /opt/rudder/etc/disable-agent |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't the enable command also check that cf-execd is running and start it if not? Otherwise, it's not really enabled, or it will be but only after some random delay...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will be run after 5mn by cron if the user forgets to run it.
I think enable just mean enable and not run, but we can add a parameter for that.
Aside from the comments above, this looks AWESOME! |
70a3e3e
to
c30431e
Compare
@jooooooon PR updated |
echo " Available commands:" | ||
for file in `cd "${BASEDIR}"; find . -type f -name "${role}-*" | sort` | ||
do | ||
doc=`grep "# @description" "${BASEDIR}/${file}" | sed -e "s/#[ \t]*@description[ \t]*//"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually meant to change the regex in both places - also in the grep :)
c30431e
to
8088292
Compare
Updated |
…and_line_for_rudder_agent Fixes #5791: Implementation of a command line for rudder agent
http://www.rudder-project.org/redmine/issues/5791